-
Notifications
You must be signed in to change notification settings - Fork 94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
release: Update go modules for pre-release #1892
release: Update go modules for pre-release #1892
Conversation
src/cloud-api-adaptor/go.mod
Outdated
@@ -47,8 +47,8 @@ require ( | |||
github.com/aws/aws-sdk-go-v2/service/eks v1.29.5 | |||
github.com/aws/aws-sdk-go-v2/service/iam v1.22.5 | |||
github.com/aws/aws-sdk-go-v2/service/s3 v1.38.5 | |||
github.com/confidential-containers/cloud-api-adaptor/src/cloud-providers v0.8.2 | |||
github.com/confidential-containers/cloud-api-adaptor/src/peerpod-ctrl v0.8.2 | |||
github.com/confidential-containers/cloud-api-adaptor/src/cloud-providers v0.9.0-alpha0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This the "alpha" usage problem that you mentioned on last meeting? i.e., caa always tagged <version>-alphaN
but now <version>
is actually an alpha release too!
If so, would make sense to call it v0.9.0-alpha1-alpha0
? I confess this seems ugly and I never saw such as versioning before...but just in case, let me ask :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I considered that myself, but thought it was ugly. If it occured to you then it's probably worth switching :)
We are working on the `v0.9.0-alpha1` pre-release, so bump the go modules to use the new tag we will create once they are merged. Due to the pre-release of an alpha the version looks messy, but we will improve it soon Signed-off-by: stevenhorsman <steven@uk.ibm.com>
d503d88
to
dffbe03
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hope never see those alpha of alpha again :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
We are working on the
v0.9.0-alpha1
pre-release, so bump the go modules to use the new tag we will create once they are merged.